Conversation
|
Are deploys still automatic on pushes to |
Deploys were not automatic on pushes to main. So we still trigger them here |
That was the primary reason we had the separate branches, previously. |
I see. We could still change the release flow to trigger automatic release once the PR is merged into The reason to make main as the default branch is to avoid keeping two branches. Currently, when a squashed commit is created into Also, currently, the flow requires someone to locally merge commits from stating into main (without squashing) and then push to main. This could lead unnecessary changes that are not reviewed in PRs. So, it's better to have only one branch where everyone's code gets reviewed and merged. |
|
@mvadari @Patel-Raj11
This way, we wouldn’t need to open PRs just to merge branches around, or manually trigger a GitHub workflow per environment (readme). We could even drop steps 3 and 4 and have every merge auto-deploy to This PR moves us one step closer to that goal. |
Yes, that makes sense. Just wanted to make sure there wasn't an auto deploy to prod on every commit to |
High Level Overview of Change
Make
mainas default branch to avoidstagingtomainPR during release. This also helps removing all the users who have push to main access and all the commits go through pull request reviews before they are merged and released.Context of Change
Type of Change
Codebase Modernization
Before / After
Test Plan